-
Notifications
You must be signed in to change notification settings - Fork 178
A serial driver should be installed on Windows 7 only #1334
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The wording is "Windows 7" rather than "earlier than Windows 8" (which may cause confusion - is it inclusive of 8?) or "Windows 7 or earlier" (I don't think we support Vista, etc.), does it sound okay? |
|
I assume we don't want to actively advertise the need of the driver but keep the content as it's searchable. So maybe references from lora, dap and possibly UnbufferedSerial/BufferedSerial could be removed. Note most mbed-os-example-*** apps use the serial port but we're not advertising the need for the driver (I don't think we should). |
|
@MarceloSalazar Completely agree. I just removed links to the serial driver doc from pages other than intro and serial communication which are directly relevant. |
MarceloSalazar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good - thanks!
| The Arm Mbed microcontroller on your board can communicate with a host PC over the same USB cable that you use for programming. | ||
|
|
||
| <span class="tips">If you're working on Windows earlier than Windows 10, you might need to [install a serial driver](../program-setup/windows-serial-driver.html).</span> | ||
| <span class="tips">If you're working on Windows 7, you might need to [install a serial driver](../program-setup/windows-serial-driver.html).</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So what about Windows 8? Does that work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it works on Windows 8.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To clarify, it shouldn't be used on Windows 8 either
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry by "works" I meant no driver needed
Co-authored-by: Irit Arkin <irit.arkin@arm.com>
* That images has been broken for ages * Adding AWS - but not to 6.0 yet because it 404s * A serial driver should be installed on Windows 7 only (#1334) * A serial driver should be installed on Windows 7 only * Remove links to serial driver from feature-related pages * Apply suggestions from code review Co-authored-by: Irit Arkin <irit.arkin@arm.com> Co-authored-by: Irit Arkin <irit.arkin@arm.com> * That doesn't work yet Co-authored-by: Lingkai Dong <lingkai.dong@arm.com>
Fixes #1332
This PR